Add crash report support to the CrashDump extension by Copilot · Pull Request #8191 · microsoft/testfx

Copilot · 2026-05-13T17:02:40Z

New Feature

What does this feature do?

The CrashDump extension can now ask the .NET runtime to generate JSON crash reports in addition to, or instead of, a dump. This makes crash triage lighter-weight in CI, especially for environments where full dump collection is expensive or impractical.

Why is this feature needed?

DOTNET_EnableCrashReport and DOTNET_EnableCrashReportOnly are already available in the runtime, but the MTP CrashDump extension only exposed dump generation. Surfacing crash reports gives a cheaper diagnostic path and improves crash investigation on machines where developers cannot inspect native dumps directly.

Implementation details

CLI surface
- Added --crashreport to generate a crash report alongside --crashdump
- Added --crashreport-only to request only the JSON crash report
- Added validation for invalid combinations:
  - --crashreport requires --crashdump
  - --crashreport-only cannot be combined with other crash options
Runtime configuration
- Wired the new options to the runtime environment variables:
  - DOTNET_EnableCrashReport
  - DOTNET_EnableCrashReportOnly
- Kept DbgEnableMiniDump enabled for --crashreport-only, since the runtime still requires createdump activation to emit the report
Artifacts and user-visible behavior
- Added crash report artifact discovery/publishing for *.crashreport.json
- Updated crash messages to distinguish:
  - dump only
  - report only
  - dump + report
- Updated help text and package docs to describe the new options
Coverage
- Extended unit coverage for command-line validation
- Added acceptance coverage for:
  - dump + crash report
  - crash report only
  - invalid --crashreport usage
- Updated help/info expectations for the new CLI options

Example

# Generate a dump and a JSON crash report
dotnet test -- --crashdump --crashreport

# Generate only a JSON crash report
dotnet test -- --crashreport-only

Co-authored-by: Evangelink <11340282+Evangelink@users.noreply.github.com>

Copilot

Pull request overview

Extends the CrashDump MTP extension with new --crashreport and --crashreport-only options that wire the test host runtime variables DOTNET_EnableCrashReport / DOTNET_EnableCrashReportOnly and publish the resulting *.crashreport.json as artifacts.

Changes:

New CLI options with mutual-exclusion validation (--crashreport requires --crashdump; --crashreport-only excludes the others).
Environment variable provider sets the new runtime variables (and keeps DbgEnableMiniDump for --crashreport-only); lifetime handler emits new "dump only / report only / dump + report" messages and publishes crash report artifacts.
New resx/xlf entries, PACKAGE.md doc update, unit/acceptance/help-info test coverage for the new options.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpCommandLineOptions.cs	Adds option name constants for the two new flags.
src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpCommandLineProvider.cs	Registers new options and adds combined-options validation via chained ternary.
src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpEnvironmentVariableProvider.cs	Sets/validates `DOTNET_EnableCrashReport(Only)` and conditionalizes `DbgEnableMiniDump`.
src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpProcessLifetimeHandler.cs	Picks crash message variant; publishes `.crashreport.json` artifacts in addition to dumps.
src/Platform/Microsoft.Testing.Extensions.CrashDump/Resources/CrashDumpResources.resx	New resource strings for descriptions, errors, and crash messages.
src/Platform/Microsoft.Testing.Extensions.CrashDump/Resources/xlf/*.xlf	Auto-added `state="new"` entries mirroring the resx additions across all locales.
src/Platform/Microsoft.Testing.Extensions.CrashDump/PACKAGE.md	Documents new crash report capabilities.
test/UnitTests/Microsoft.Testing.Extensions.UnitTests/CrashDumpTests.cs	Adds unit tests for valid/invalid combinations of the new flags.
test/IntegrationTests/Microsoft.Testing.Platform.Acceptance.IntegrationTests/CrashDumpTests.cs	Acceptance tests for dump+report, report-only, and `--crashreport` without `--crashdump`.
test/IntegrationTests/Microsoft.Testing.Platform.Acceptance.IntegrationTests/HelpInfoAllExtensionsTests.cs	Updates expected help/info output to include the new options.

Copilot's findings

Files reviewed: 22/22 changed files
Comments generated: 3

Evangelink · 2026-05-14T18:51:19Z

+        => commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOptionName) &&
+            !commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashDumpOptionName)
+            ? ValidationResult.InvalidTask(CrashDumpResources.CrashReportRequiresCrashDumpErrorMessage)
+            : commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOnlyOptionName) &&
+            (commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashDumpOptionName) ||
+             commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOptionName))
+            ? ValidationResult.InvalidTask(CrashDumpResources.CrashReportOnlyCannotBeCombinedErrorMessage)
+            : ValidationResult.ValidTask;


Done in e8b4818. Refactored into explicit if-statements (one per invalid combination) so future mutually-exclusive crash options are easy to add.

Evangelink · 2026-05-14T18:51:20Z

+        if (crashReportEnabled)
+        {
+            foreach (string prefix in Prefixes)
+            {
+                environmentVariables.SetVariable(new($"{prefix}{EnableCrashReportVariable}", EnableMiniDumpValue, false, true));
+            }
+        }
+
+        if (crashReportOnlyEnabled)
+        {
+            foreach (string prefix in Prefixes)
+            {
+                environmentVariables.SetVariable(new($"{prefix}{EnableCrashReportOnlyVariable}", EnableMiniDumpValue, false, true));
+            }
+        }


Done in e8b4818. Renamed EnableMiniDumpValue to the neutral EnabledValue since it's now also used for DOTNET_EnableCrashReport and DOTNET_EnableCrashReportOnly.

Evangelink · 2026-05-14T18:51:22Z

+        TestHostResult testHostResult = await testHost.ExecuteAsync($"--crashreport-only --crashdump-filename customdumpname.dmp --results-directory {resultDirectory}", cancellationToken: TestContext.CancellationToken);
+        testHostResult.AssertExitCodeIs(ExitCode.TestHostProcessExitedNonGracefully);
+
+        Assert.IsEmpty(Directory.GetFiles(resultDirectory, "customdumpname.dmp", SearchOption.AllDirectories), $"Unexpected dump file found\n{testHostResult}");


Done in e8b4818. Switched to Directory.EnumerateFiles(*).Where(f => string.Equals(Path.GetFileName(f), "customdumpname.dmp", StringComparison.Ordinal)) so the assertion isn't tricked by Windows pattern-matching matching customdumpname.dmp.crashreport.json.

…shDump tests/code - Reorder --crashreport / --crashreport-only after --crashdump-type in the Help and Info expectations to match the platform's alphabetical ordering (CommandLineHandler.PrintOptionsAsync OrderBy(option.Name)). - Refactor CrashDump option validation into explicit if-statements for clarity. - Rename EnableMiniDumpValue -> EnabledValue (now reused for crash report vars). - Make CrashReportOnly_CustomDumpName_CreateOnlyCrashReport robust on Windows by doing an exact filename comparison instead of relying on Directory.GetFiles pattern matching, which can match 'customdumpname.dmp.crashreport.json'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Revert the validation refactor, EnabledValue rename, and the Windows-safe filename check in CrashReportOnly_CustomDumpName_CreateOnlyCrashReport. The HelpInfoAllExtensionsTests ordering fix is preserved. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Copilot's findings

Comments suppressed due to low confidence (1)

src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpProcessLifetimeHandler.cs:1

Mirror of the existing dump-fallback path: if Path.GetDirectoryName(expectedCrashReportFile) returns an empty string (when the dump file pattern is just a filename with no directory component), Directory.GetFiles("", ...) will throw ArgumentException. The pre-existing dump branch has the same issue, but consider falling back to the current directory or skipping the scan when the directory is empty so the new code path doesn't introduce another instance of the same brittleness.

// Copyright (c) Microsoft Corporation. All rights reserved.

Files reviewed: 22/22 changed files
Comments generated: 8

+        => commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOptionName) &&
+            !commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashDumpOptionName)
+            ? ValidationResult.InvalidTask(CrashDumpResources.CrashReportRequiresCrashDumpErrorMessage)
+            : commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOnlyOptionName) &&
+            (commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashDumpOptionName) ||
+             commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOptionName))
+            ? ValidationResult.InvalidTask(CrashDumpResources.CrashReportOnlyCannotBeCombinedErrorMessage)
+            : ValidationResult.ValidTask;


+    <value>[net6.0+ only] Generate a crash report file if the test process crashes. Requires '--crashdump'.</value>
+  </data>
+  <data name="CrashReportRequiresCrashDumpErrorMessage" xml:space="preserve">
+    <value>You specified '--crashreport' but did not enable crash dumps, add --crashdump to the command line</value>


+        if (crashDumpEnabled || crashReportOnlyEnabled)
+        {
+            foreach (string prefix in Prefixes)
+            {
+                environmentVariables.SetVariable(new($"{prefix}{EnableMiniDumpVariable}", EnableMiniDumpValue, false, true));
+            }
+        }


        {
-            await _outputDisplay.DisplayAsync(this, new ErrorMessageOutputDeviceData(string.Format(CultureInfo.InvariantCulture, CrashDumpResources.CannotFindExpectedCrashDumpFile, expectedDumpFile)), cancellationToken).ConfigureAwait(false);
-            foreach (string dumpFile in Directory.GetFiles(Path.GetDirectoryName(expectedDumpFile)!, "*.dmp"))
+            string expectedCrashReportFile = $"{expectedDumpFile}.crashreport.json";


+    }
+
+    [TestMethod]
+    public async Task CrashReportOnly_CustomDumpName_CreateOnlyCrashReport()


+        if (_commandLineOptions.IsOptionSet(CrashDumpCommandLineOptions.CrashReportOptionName))
+        {
+            foreach (string prefix in Prefixes)
+            {
+                if (!environmentVariables.TryGetVariable($"{prefix}{EnableCrashReportVariable}", out OwnedEnvironmentVariable? enableCrashReport)
+                || enableCrashReport.Value != EnableMiniDumpValue)
+                {
+                    AddError(errors, $"{prefix}{EnableCrashReportVariable}", EnableMiniDumpValue, enableCrashReport?.Value);


+    [TestMethod]
+    public async Task CrashReportOnly_Cannot_Be_Combined_With_Other_Crash_Options()
+    {
+        var provider = new CrashDumpCommandLineProvider();
+        var options = new Dictionary<string, string[]>
+        {
+            { CrashDumpCommandLineOptions.CrashDumpOptionName, [] },


+        if (crashReportEnabled)
+        {
+            foreach (string prefix in Prefixes)
+            {
+                environmentVariables.SetVariable(new($"{prefix}{EnableCrashReportVariable}", EnableMiniDumpValue, false, true));
+            }
+        }


- CrashDumpCommandLineProvider: replace chained ternary with explicit if-statements for clarity and easier extension. - CrashDumpEnvironmentVariableProvider: rename EnableMiniDumpValue to EnabledValue since it's now reused for the crash report environment variables (DOTNET_EnableCrashReport / DOTNET_EnableCrashReportOnly). - CrashReportOnly_CustomDumpName_CreateOnlyCrashReport: do an explicit filename comparison instead of relying on Directory.GetFiles' pattern matching, which on Windows can also match 'customdumpname.dmp.crashreport.json' for the literal pattern 'customdumpname.dmp'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Evangelink · 2026-05-14T18:51:26Z

@copilot address review comments

Co-authored-by: Evangelink <11340282+Evangelink@users.noreply.github.com>

Copilot · 2026-05-14T19:01:57Z

@copilot address review comments

Addressed in d68feac. I covered the remaining CrashDump review feedback: validation precedence/coverage, resource quoting and XLF sync, crash report minidump wiring, crash report suffix constants, environment validation deduplication, and the acceptance test rename.

Initial plan

fa259c4

Copilot AI self-assigned this May 13, 2026

Copilot AI review requested due to automatic review settings May 13, 2026 17:02

Copilot AI assigned Evangelink May 13, 2026

Copilot AI review requested due to automatic review settings May 13, 2026 17:02

Copilot started work on behalf of Evangelink May 13, 2026 17:02 View session

Copilot AI linked an issue May 13, 2026 that may be closed by this pull request

Feature request: Generate crash report #7168

Open

Add crash report support to crash dump extension

b904e02

Co-authored-by: Evangelink <11340282+Evangelink@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 13, 2026 17:23

Copilot AI changed the title ~~[WIP] Add feature to generate crash report for .NET 6.0 and 7.0~~ Add crash report support to the CrashDump extension May 13, 2026

Copilot AI requested a review from Evangelink May 13, 2026 17:24

Copilot finished work on behalf of Evangelink May 13, 2026 17:24

Evangelink marked this pull request as ready for review May 14, 2026 14:59

Copilot AI review requested due to automatic review settings May 14, 2026 14:59

Merge branch 'main' into copilot/feature-generate-crash-report

8882424

Copilot started reviewing on behalf of Evangelink May 14, 2026 15:02 View session

Copilot AI reviewed May 14, 2026

View reviewed changes

Amaury Levé and others added 2 commits May 14, 2026 20:19

Copilot AI review requested due to automatic review settings May 14, 2026 18:29

Copilot AI reviewed May 14, 2026

View reviewed changes

Copilot started work on behalf of Evangelink May 14, 2026 18:51 View session

Address CrashDump review comments

d68feac

Co-authored-by: Evangelink <11340282+Evangelink@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 14, 2026 19:01

Copilot finished work on behalf of Evangelink May 14, 2026 19:02

Copilot started reviewing on behalf of Evangelink May 14, 2026 19:02 View session

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add crash report support to the CrashDump extension#8191

Add crash report support to the CrashDump extension#8191
Copilot wants to merge 7 commits into
mainfrom
copilot/feature-generate-crash-report

Copilot AI commented May 13, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Evangelink May 14, 2026

Uh oh!

Evangelink May 14, 2026

Uh oh!

Evangelink May 14, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Evangelink commented May 14, 2026

Uh oh!

Copilot AI commented May 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented May 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New Feature

What does this feature do?

Why is this feature needed?

Implementation details

Example

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Copilot's findings

Uh oh!

Evangelink May 14, 2026

Choose a reason for hiding this comment

Uh oh!

Evangelink May 14, 2026

Choose a reason for hiding this comment

Uh oh!

Evangelink May 14, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot's findings

Uh oh!

Evangelink commented May 14, 2026

Uh oh!

Copilot AI commented May 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented May 13, 2026 •

edited

Loading